Skip to content

refactor(css): narrow home-shell overrides so Codex owns native chrome - #293

Open
starsstreaming wants to merge 4 commits into
Fei-Away:mainfrom
starsstreaming:feat/css-narrow-native-surfaces
Open

refactor(css): narrow home-shell overrides so Codex owns native chrome#293
starsstreaming wants to merge 4 commits into
Fei-Away:mainfrom
starsstreaming:feat/css-narrow-native-surfaces

Conversation

@starsstreaming

Copy link
Copy Markdown

Summary / 摘要

Stacked on the verified-rollback PR (and transitively atomic + #290).

Narrow Dream Skin's home/shell CSS so Codex keeps ownership of native chrome: stop clearing bg-token-main-surface-primary panes, stop rewriting composer radius/border/shadow/backdrop, and stop forcing utility-bar z-index / translateY. Video stage rules from #290 stay. Additive skin layers that do not fight native chrome remain.

叠在 verified-rollback(及原子写入 + #290)之上。收窄首页/主壳 CSS:不再清空原生 token surface、不再改写 composer 几何与 utility bar 层叠;保留 #290 视频舞台规则

Depends on / 依赖: previous stacked PRs. Review base: starsstreaming:feat/verified-theme-rollback.

Overlap warning / 与开放 PR 重叠说明

Touches the same CSS files as open #177 (panel translucency / artwork stability) and #263. This PR removes global token-surface clearing and composer chrome overrides — the opposite direction of keeping panel translucency. Prefer merging or reconciling with #177/#263 before or with this PR. Happy to adjust if maintainers want a different split.

与开放 PR #177#263 改同一批 CSS。本 PR 删除全局 token surface 透明化与 composer 外观覆盖,与「保持面板半透明」方向相反。建议与 #177/#263 协调后再合。

Type / 类型

  • Bug fix / 缺陷修复
  • Feature / 新功能
  • Docs / 文档
  • Theme / CSS / visual / 主题或视觉
  • Scripts / install / restore / 脚本或安装恢复
  • Chore / 杂项

Platform / 平台

  • macOS
  • Windows
  • Both / 双平台
  • Docs / repo only / 仅文档或仓库元数据

Self-check / 自测

macOS

  • macos/tests/run-tests.sh — not run on Windows host
  • Live verify — pending screenshots / Mac

Windows

  • node --test tools/renderer-runtime.test.mjs — pass
  • node tools/sync-runtime-assets.mjs — exit 0
  • Environment noted below

User-facing

  • Updated macos/CHANGELOG.md and windows/CHANGELOG.md under ## 未发布 (no VERSION bump)

Security / 安全

  • Does not modify official Codex install / asar / signatures
  • Does not silently write API Base URL or keys
  • CDP remains loopback-oriented (127.0.0.1)

Notes / 补充

Diff shape

  • Pure-delete / comment-replace on composer + token-surface + utility-bar rules (~-126 lines x runtime + 2 asset copies)
  • Keeps #codex-dream-skin-background-stage / data-dream-media="video" block
  • Regression doesNotMatch asserts in tools/renderer-runtime.test.mjs

Screenshots

Visual change — homepage/task stills without private chat can be attached after open if reviewers want frame captures (video demo already on #290).

Stack

  1. feat(runtime): add cross-platform MP4 video backgrounds #290 video
  2. atomic marker
  3. verified tray rollback
  4. this PR CSS narrow

Environment

  • Windows 11 Pro 10.0.26200
  • Branch: starsstreaming:feat/css-narrow-native-surfaces @ f196381

Support optional theme.json video fields, loopback media serving,
menu/tray import, poster fallback, and renderer handoff on both
platforms. Closes Fei-Away#240.
Write .theme-update-in-progress only around the final image/video/json/css
commit so the watcher cannot load a mixed stamp, then clear it on success.
Stale markers expire so a crashed writer cannot freeze theme loads forever.
Snapshot the active theme before image, video, or saved-theme tray
switches; confirm on a real Codex window; restore the previous theme
when verification fails. Move snapshot helpers into theme-windows and
rename the serialized tray helper parameter to Operation so nested
verified actions cannot recurse into themselves.
Stop clearing token-surface backgrounds and stop rewriting composer
radius, border, shadow, backdrop, and utility-bar stacking. Keep the
video stage rules and other skin layers that do not fight native chrome.
@starsstreaming

Copy link
Copy Markdown
Author

Stack: depends on #292 (+ #291 + #290). Overlap note with open #177 / #263 is in the PR body.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant